Conversation
✅ Deploy Preview for hyprnote canceled.
|
✅ Deploy Preview for char-cli-web canceled.
|
✅ Deploy Preview for unsigned-char ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 6b54113. Configure here.
| participantsJson: | ||
| event.participants.length > 0 | ||
| ? JSON.stringify(event.participants) | ||
| : undefined, |
There was a problem hiding this comment.
Double JSON stringification corrupts participantsJson in database
High Severity
The participantsJson column is defined in the Drizzle schema with text("participants_json", { mode: "json" }), which means Drizzle automatically calls JSON.stringify() on inserts/updates. However, executeForEventsSync pre-stringifies with JSON.stringify(event.participants) before passing to updateEvent/insertEvent, causing double-encoding. The stored value becomes an escaped JSON string (e.g., "[{\"name\":\"John\"}]") instead of a proper JSON array. The existing templates code in this codebase correctly passes raw objects to JSON-mode columns, confirming the expected pattern.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 6b54113. Configure here.
| "activity_screenshots", | ||
| "daily_notes", | ||
| "daily_summaries", | ||
| "prompt_overrides", |
There was a problem hiding this comment.
Migration test missing new calendars and events tables
Medium Severity
The migration_creates_tables test asserts an expected table list that doesn't include calendars or events, even though the new migration 20260414120000_calendars_events.sql creates both tables. The test queries sqlite_master after running all migrations, so these tables will exist, causing the assertion to fail.
Reviewed by Cursor Bugbot for commit 6b54113. Configure here.
1 participant
No description provided.